Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

test/ui-e2e/tests/admin-login.spec.ts (1)

30-52: ⚡ Quick win

Ensure browser context is always closed via finally.

If any step throws before Line 51, the manually created context can remain open and make UI suites flaky/resource-heavy. Wrap the test body after context creation in try/finally so cleanup always runs.

Proposed patch

   //Fresh context to avoid any cached state issues
   const context = await browser.newContext({ 
     storageState: { cookies: [], origins: [] },
     ignoreHTTPSErrors: true 
   });
-  
-  //Navigate and wait for the page to be loaded
-  const page = await context.newPage();
-  const loginUrl = `https://${routeUrl}/login?dex=none`;
-  await page.goto(loginUrl, { waitUntil: 'load' });
-
-  const userField = page.getByLabel(/username/i);
-  await userField.waitFor({ state: 'visible', timeout: 20000 });
-
-  //Fill and Sign In
-  await userField.fill('admin');
-  await page.locator('input[type="password"]').fill(password);
-  await page.getByRole('button', { name: /sign in/i }).click();
-
-  //Verify we're logged in
-  await expect(page.locator('.sidebar, [data-testid="sidebar"]').first()).toBeVisible({ timeout: 20000 });
-
-  await context.close();
+  try {
+    //Navigate and wait for the page to be loaded
+    const page = await context.newPage();
+    const loginUrl = `https://${routeUrl}/login?dex=none`;
+    await page.goto(loginUrl, { waitUntil: 'load' });
+
+    const userField = page.getByLabel(/username/i);
+    await userField.waitFor({ state: 'visible', timeout: 20000 });
+
+    //Fill and Sign In
+    await userField.fill('admin');
+    await page.locator('input[type="password"]').fill(password);
+    await page.getByRole('button', { name: /sign in/i }).click();
+
+    //Verify we're logged in
+    await expect(page.locator('.sidebar, [data-testid="sidebar"]').first()).toBeVisible({ timeout: 20000 });
+  } finally {
+    await context.close();
+  }
 });

As per coding guidelines, "-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test/ui-e2e/tests/admin-login.spec.ts` around lines 30 - 52, The test creates
a Playwright context via browser.newContext (variable context) but may exit
early and never close it; wrap the steps after creating context (newContext,
newPage, navigation, interactions, assertions) in a try/finally and call await
context.close() in the finally block to guarantee cleanup even on failure;
locate the context/newPage usage and the final await context.close() in the
admin-login.spec.ts test and move the close into the finally to ensure resources
are always released.